Skip to content

fix(wallet): add optimistic holder count update on key purchase (#780) - #805

Closed
Cmitchelle7 wants to merge 1 commit into
accesslayerorg:mainfrom
Cmitchelle7:fix/optimistic-holder-count-780
Closed

fix(wallet): add optimistic holder count update on key purchase (#780)#805
Cmitchelle7 wants to merge 1 commit into
accesslayerorg:mainfrom
Cmitchelle7:fix/optimistic-holder-count-780

Conversation

@Cmitchelle7

Copy link
Copy Markdown
Contributor

Description
Adds optimistic holder count increment on key purchase using React Query's onMutate callback.

Changes

  • onMutate: increments ['creator', creatorId, 'holderCount'] cache by 1
  • onError: rolls back holder count to previous value
  • onSettled: invalidates holder count query to sync with server

Closes #780

@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

@Cmitchelle7 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@netlify

netlify Bot commented Aug 26, 2026

Copy link
Copy Markdown

Deploy Preview for accesslayer failed.

Name Link
🔨 Latest commit 30d5161
🔍 Latest deploy log https://app.netlify.com/projects/accesslayer/deploys/6a8f1be587d83f0008649a31

@github-actions

Copy link
Copy Markdown

👋 Hey @Cmitchelle7, thanks for your contribution!

This PR is targeting main directly. We use main for stable releases only — all contributions should be opened against the dev branch instead.

Please close this PR and reopen it with dev as the base branch. If you're unsure how to do that, you can change the base branch using the Edit button at the top of this PR page.

Closing this PR automatically. See you in dev! 🚀

@github-actions github-actions Bot closed this Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant